Skip to content

fix(sandbox): avoid FUSE permission failures - #90

Merged
iamjr15 merged 1 commit into
mainfrom
fix/fuse-upload-permissions
Jul 28, 2026
Merged

fix(sandbox): avoid FUSE permission failures#90
iamjr15 merged 1 commit into
mainfrom
fix/fuse-upload-permissions

Conversation

@iamjr15

@iamjr15 iamjr15 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove Daytona ownership/mode calls from the persistent FUSE-backed workspace path
  • retain the separate workspace-materialization record, R2 checksum repair, bounded file-plane verification, and application-level write/delete guards
  • publish the runtime manifest directly through the toolbox file plane without a shell visibility dependency
  • document the actual enforcement boundary instead of claiming portable FUSE permissions

Production finding

The deployed permissions implementation returned Daytona HTTP 403 while repairing a deliberately removed workspace cache file. That failure occurred before model access and the same call failed again during cleanup. The mounted volume cannot use the provider permissions endpoint as a reliable ownership boundary, so allowing it to fail the user run is incorrect.

Verification

  • pnpm turbo lint typecheck build — 76/76 tasks passed on Node 24.18.0
  • affected architecture boundary check — passed, 113 modules / 210 dependencies
  • affected Knip workspaces — passed
  • git diff --check — passed
  • exactly one Drizzle migration remains; no database changes

Acceptance after merge

Repeat the same destructive cache-only test: remove uploads/README.md, verify exact R2 checksum recovery before the agent reads it, verify the runtime manifest exists, and ensure a following unchanged run does not rematerialize the file.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
cheatcode Skipped Skipped Jul 28, 2026 4:50am

@iamjr15
iamjr15 merged commit 33a3b18 into main Jul 28, 2026
6 checks passed
@iamjr15
iamjr15 deleted the fix/fuse-upload-permissions branch July 28, 2026 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant